Skip to content

Disable Triton Due to Compatibility Issues#1020

Merged
sayanshaw24 merged 7 commits intomainfrom
sayanshaw/rapidjson
Feb 4, 2026
Merged

Disable Triton Due to Compatibility Issues#1020
sayanshaw24 merged 7 commits intomainfrom
sayanshaw/rapidjson

Conversation

@sayanshaw24
Copy link
Collaborator

@sayanshaw24 sayanshaw24 commented Feb 3, 2026

Updates

This PR disables triton in official builds by default, which is the HTTP client for the AzureTritonInvoker operator that lets ONNX models call Azure ML Triton inference endpoints.

The linux packaging pipeline was failing due to triton build issues and cascading failures when building with manylinux_2_28 (AlmaLinux 8) and GCC 14.2.1/CMake 4.2, where:

  • RapidJSON's cmake_minimum_required(VERSION 2.8) fails with CMake 4.2
  • RapidJSON 1.1.0 has C++ compatibility issues with GCC 14
  • Similar issues

Validation

  • Ran Linux packaging pipeline successfully

@sayanshaw24 sayanshaw24 changed the title Add triton and rapidjson patch Disable Triton Due to Compatibility Issues Feb 4, 2026
@sayanshaw24 sayanshaw24 marked this pull request as ready for review February 4, 2026 19:21
@sayanshaw24 sayanshaw24 requested a review from a team as a code owner February 4, 2026 19:21
@sayanshaw24 sayanshaw24 enabled auto-merge (squash) February 4, 2026 19:21
@sayanshaw24 sayanshaw24 merged commit 606591f into main Feb 4, 2026
37 checks passed
@sayanshaw24 sayanshaw24 deleted the sayanshaw/rapidjson branch February 4, 2026 20:37
sayanshaw24 added a commit that referenced this pull request Feb 4, 2026
* add triton and rapidjson patch

* add support for AlmaLinux and Rocky Linux

* fix GCC 14 compatibility issues

* use RapidJSON master with GCC 14 fixes and backward compat patch

* use RapidJSON master with GCC 14 fixes and backward compat patch

* remove patch

* disable triton

---------

Co-authored-by: Sayan Shaw <sayanshaw@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants